Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: zerone0x <hi@trine.dev>

- codebase more clear for humans and AI agents
- alignement on vision and filter quality in technical docs

…folders-and-documentation

Signed-off-by: Ахмедов Нурбек Медетбаевич <n.akhmedov@uzdc.uz>

…folders-and-documentation

feat(refacto-codebase-onboarding): partie 1 - folders and technical docs

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

rtk ls | wc -l returned N+2 instead of N because the summary line
(📊 X files, Y dirs) and blank line were always written to stdout.

Now uses std::io::stdout().is_terminal() to suppress the summary
when piped, following standard Unix convention (ls, grep, find all
do this).

Fixes #482

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Ousama Ben Younes <benyounes.ousama@gmail.com>

Avoids one allocation since entries is not used after the if/else.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Ousama Ben Younes <benyounes.ousama@gmail.com>

Fixes Documentation Validation CI check (main.rs=64, ARCHITECTURE.md was 60).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Ousama Ben Younes <benyounes.ousama@gmail.com>

Signed-off-by: Roopesh <roopesh1724989@gmail.com>

cat flags like -v, -A, -e, -t, -s have different semantics than
rtk read flags (-v means verbose, -n means line numbers). Blindly
forwarding flags caused incorrect behavior (e.g. cat -A file →
rtk read -A file which fails).

Only skip rewrite for incompatible flags. cat -n (line numbers) maps
correctly to rtk read -n, so it is still rewritten for token savings.
Plain `cat file` continues to rewrite to `rtk read file` as before.

As of go v1.24, tools such as golanci-lint can be managed via the mod
file and called directly on the go binary via the tool subcommand.

When called via tools, we now apply the same logic to compact the
results as in the directly called command.

Signed-off-by: shwoop <monkey.mambo.al.ferguson@gmail.com>

fix(ls): suppress summary line when stdout is piped

LGTM. Correct fix — Clap's command graph can overflow the default 1MB Windows stack. /STACK:8388608 is a standard mitigation.

Docs correct — branch names use slash prefixes, not conventional-commit format in parentheses.

Correct fix. disable_help_flag = true is the right pattern for passthrough commands — consistent with other commands in the codebase.

…n-cargo-output

Correct. The Finished line carries useful signal (build time + profile). Defensive implementation with Option fallback.

Standard portability fix. /bin/bash is not guaranteed on NixOS or macOS with Homebrew bash. /usr/bin/env bash is the correct practice.

…or-details

LGTM after code review. truncate() correctly imported, error_details block placed before rule frequency table, test covers the main case. Minor duplication for error[Exxxx] style but not a bug.

Correct fix. cat flags like -A/-v/-e have no rtk read equivalent — skipping rewrite is the right behavior. cat -n correctly preserved. Well tested.

feat(go): Support golangci-lint via go tool

fix(security): salt device hash for telemetry